Skip to content

Append the output of grad_transform_on_trace to computation_traces - #2392

Merged
t-vi merged 3 commits into
mainfrom
crcrpar-patch-3
Sep 9, 2025
Merged

t-vi merged 3 commits into
mainfrom
crcrpar-patch-3

Conversation

@crcrpar

@crcrpar crcrpar commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator

What does this PR do?

It'd be reasonable to append the output of grad_transform_on_trace to computation_traces: list[TraceCtx]

@crcrpar
crcrpar requested a review from beverlylytle August 1, 2025 18:08

@beverlylytle beverlylytle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

It'd be reasonable to append the output of `grad_transform_on_trace` to `computation_traces: list[TraceCtx]`

@t-vi t-vi left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @crcrpar

@t-vi
t-vi enabled auto-merge (squash) August 27, 2025 08:05
@t-vi
t-vi merged commit cc45a5c into main Sep 9, 2025
68 of 73 checks passed
@t-vi
t-vi deleted the crcrpar-patch-3 branch September 9, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants